Skip to content

Conversation

@millotp
Copy link
Collaborator

@millotp millotp commented Sep 20, 2024

🧭 What and Why

Don't run benchmark in race mode because it kills the perfs

@millotp millotp requested a review from a team as a code owner September 20, 2024 12:17
@millotp millotp requested review from Fluf22 and shortcuts September 20, 2024 12:17
@algolia-bot
Copy link
Collaborator

algolia-bot commented Sep 20, 2024

✔️ Code generated!

Name Link
🪓 Triggered by 718047eee7dea759c1ca13409d346e0340ae1e0f
🍃 Generated commit 33ef0b657145d59b077f2d2e3db96eea7074d399
🌲 Generated branch generated/fix/fast-go
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Req/s
javascript 1851
go 1713
php 1561
csharp 1330
python 1087
java 1081
ruby 883
swift 766

@shortcuts
Copy link
Member

ooooh that will be FAST

@shortcuts
Copy link
Member

(btw this is a chore we shouldn't put it in the changelog)

shortcuts
shortcuts previously approved these changes Sep 20, 2024
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nothing to add! waiting for the gen, it looks great

@millotp
Copy link
Collaborator Author

millotp commented Sep 20, 2024

(btw this is a chore we shouldn't put it in the changelog)

The name of the commit doesn't determine the changelog anymore, only if there are changes in the clients folder.
I guess this should be a fix(benchmark) maybe

@millotp millotp requested a review from shortcuts September 20, 2024 13:52
@millotp millotp enabled auto-merge (squash) September 20, 2024 16:16
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(btw this is a chore we shouldn't put it in the changelog)

The name of the commit doesn't determine the changelog anymore, only if there are changes in the clients folder. I guess this should be a fix(benchmark) maybe

true but still a chore IMO, non blocking anyway, great fix!

@millotp millotp merged commit 7cde7be into main Sep 22, 2024
28 checks passed
@millotp millotp deleted the fix/fast-go branch September 22, 2024 20:02
algolia-bot added a commit that referenced this pull request Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants